Skip to content

Conversation

@arayaryoma
Copy link
Contributor

@arayaryoma arayaryoma commented May 17, 2018

Hello.
I'm using react-native-gifted-chat with TypeScript. I want messages property of GiftedChat component to have a type.
I considered to write new Interface but found IMessage type in the type definition file. It looks good to use as type of messages property.
Thanks.

@codecov
Copy link

codecov bot commented May 17, 2018

Codecov Report

Merging #861 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #861   +/-   ##
=======================================
  Coverage   43.17%   43.17%           
=======================================
  Files          20       20           
  Lines         498      498           
  Branches      104      104           
=======================================
  Hits          215      215           
  Misses        216      216           
  Partials       67       67

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update cbbf812...f608ee8. Read the comment docs.

@xcarpentier xcarpentier merged commit 1ba8f8b into FaridSafi:master Jul 9, 2018
chilinh added a commit to chilinh/react-native-gifted-chat that referenced this pull request Jul 18, 2018
* r_master:
  Use IMessage type instead of `any` to define the type of message property (FaridSafi#861)
  chore(package): update prop-types to version 15.6.2 (FaridSafi#907)
  fix(package): update uuid to version 3.3.0 (FaridSafi#913)
  Update InputToolbar.js (FaridSafi#906)
  Improved typings (FaridSafi#916)
  fix: enable renderFooter for FlatList MessageContainer (FaridSafi#914)

# Conflicts:
#	index.d.ts
jessestuart pushed a commit to jessestuart/react-native-gifted-chat that referenced this pull request Aug 9, 2018
@xcarpentier
Copy link
Collaborator

#1190

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants